Skip to content
This repository was archived by the owner on Aug 1, 2025. It is now read-only.

fix: resolve issue with artifacts naming - #8195

Merged
ichim-david merged 2 commits into
mainfrom
BolajiOlajide-patch-1
Jul 23, 2025
Merged

fix: resolve issue with artifacts naming #8195
ichim-david merged 2 commits into
mainfrom
BolajiOlajide-patch-1

Conversation

@BolajiOlajide

Copy link
Copy Markdown
Contributor

When a failure occurs in the e2e tests, we upload some of the artifacts for access. However, the naming of this artifact isn't unique and when an artifact with the same name exists, it throws an error.

This PR makes the artifact name unique and overwrites it in the event that running the workflow multiple times returns an error

Test plan

  • CI

@BolajiOlajide
BolajiOlajide requested a review from 0xnmn July 23, 2025 19:25
@BolajiOlajide BolajiOlajide self-assigned this Jul 23, 2025
@ichim-david
ichim-david merged commit f4280dc into main Jul 23, 2025
20 of 21 checks passed
@ichim-david
ichim-david deleted the BolajiOlajide-patch-1 branch July 23, 2025 20:12
@0xnmn
0xnmn restored the BolajiOlajide-patch-1 branch July 24, 2025 07:41
0xnmn pushed a commit that referenced this pull request Jul 24, 2025
When a failure occurs in the e2e tests, we upload some of the artifacts
for access. However, the naming of this artifact isn't unique and
when an artifact with the same name exists, it throws an error.

This PR makes the artifact name unique and overwrites it in the event
that running the workflow multiple times returns an error

## Test plan


* CI <br> Backport f4280dc from #8195

Co-authored-by: Bolaji Olajide <25608335+BolajiOlajide@users.noreply.github.com>
Co-authored-by: David Ichim <ichim.david@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants